Agrega comando /all para mencionar a todos los miembros del grupo - #238
Merged
Conversation
- Habilita los handlers de entrada/salida de miembros del grupo (estaban comentados) para que se registren en users/users_group a medida que la gente entra o sale, y corrige add_group para que sea async (requerido por python-telegram-bot 22) e ignore al bot mismo cuando se lo agrega a un grupo. - Agrega MainController.get_group_members(cid) para leer los miembros conocidos de un grupo. - Agrega /all, que menciona a todos los miembros conocidos del grupo (los vistos entrar, o los que se unieron a una partida vía /join, que ya registraba esto para cubrir a quienes entraron antes de este cambio). Co-Authored-By: Claude Sonnet 5 <noreply@anthropic.com> Claude-Session: https://claude.ai/code/session_014DChTLjajSCpdzg2cWA2Am
This file contains hidden or bidirectional Unicode text that may be interpreted or compiled differently than what appears below. To review, open the file in an editor that reveals hidden Unicode characters.
Learn more about bidirectional Unicode characters
Sign up for free
to join this conversation on GitHub.
Already have an account?
Sign in to comment
2 participants
Add this suggestion to a batch that can be applied as a single commit.This suggestion is invalid because no changes were made to the code.Suggestions cannot be applied while the pull request is closed.Suggestions cannot be applied while viewing a subset of changes.Only one suggestion per line can be applied in a batch.Add this suggestion to a batch that can be applied as a single commit.Applying suggestions on deleted lines is not supported.You must change the existing code in this line in order to create a valid suggestion.Outdated suggestions cannot be applied.This suggestion has been applied or marked resolved.Suggestions cannot be applied from pending reviews.Suggestions cannot be applied on multi-line comments.Suggestions cannot be applied while the pull request is queued to merge.Suggestion cannot be applied right now. Please check back later.
Summary
NEW_CHAT_MEMBERS/LEFT_CHAT_MEMBER), que estaban comentados, para que se sigan registrando enusers/users_groupa medida que la gente entra o sale del grupo. De paso corrigeadd_grouppara que seaasync(requerido porpython-telegram-bot22 — condefnormal la app iba a fallar al invocarlo) y hace que ignore al bot mismo cuando lo agregan a un grupo.MainController.get_group_members(cid), que lee de la base los miembros conocidos de un grupo./all, que menciona a todos los miembros conocidos del grupo (los que el bot vio entrar, más los que se unieron a una partida vía/join— ese flujo ya registraba enusers/users_group, así que cubre a quienes ya estaban en el grupo antes de este cambio en cuanto usen/join).Test plan
python -m py_compile Commands.py MainController.pycommand_allprobado conMainController.get_group_memberssimulado: caso con miembros, caso sin miembros guardados, y caso fuera de un grupo.add_groupprobado con un update simulado denew_chat_membersmixto (humano + bot), verificando que es coroutine y que ignora al bot.🤖 Generated with Claude Code
https://claude.ai/code/session_014DChTLjajSCpdzg2cWA2Am
Generated by Claude Code